Tim Haines

Results 183 comments of Tim Haines

@mxz297 I'm having trouble getting the test suite to run consistently on coriander, but looking at the diff of regressions between the last three runs, it looks like everything is...

In Capstone, the detailed register information (e.g., read/write) is not available if it is compiled with CAPSTONE_DIET. We need a way to detect this during Dyninst's compilation.

I'm waiting to hear back from the Capstone devs about their integration of LLVM 10. If we can work around that, then we'll just update their tables directly from Xed....

I'm running the head of master on coriander now for a baseline. I'll try to run this PR tomorrow, assuming no major issues. Could you run the build script on...

@mxz297 They merged your changes! https://github.com/capstone-engine/capstone/pull/1604 We should rebase here and set up a testing environment. The best environment would be somewhere with the latest AVX512. Maybe Arcticus or Iris/Yarrow...

~~@mxz297 The instruction 'vpperm' used to be in Dyninst, but isn't after this PR. That opcode is in Capstone, so it might be a processing error. Could you double check?~~...

We haven't test gcc-12 or glibc-2.33+ yet, so these are good to have. I have a feeling the DWARF errors are from a lack of DWARF5 support. Is your environment...

Oh, I just noticed that gcc-12 isn't public yet. Do you have any indication of when there might be a release? I'd rather test against the official binaries rather than...

If `batching`, it's a use-after-free. Otherwise, it's a double-free. The [`BlockPoints::~BlockPoints`](https://github.com/dyninst/dyninst/blob/master/patchAPI/src/Point.C#L257) also tries to delete the objects that are being deleted on PatchBlock.C:452. This function is chock full of badness....

This should be bumped up. Those high-priority items could be part of the trouble with building on Zeroah.