[PATCH] ARM: Windows on ARM Hazard recognition support

Tim Northover t.p.northover at gmail.com
Wed Apr 2 13:07:04 PDT 2014


Hi Saleem,

>   Hah, that was the first approach that I took.  The problem ended up being that the psuedo-expansion occurred too early, and the instruction was still split up :-(.  If there is a generic way to handle all the possible cases, that would most certainly be the preferred.

Do you remember any examples of instructions in bundles being split?
Because my natural instinct is to say that would be a bug, regardless
of whether it's exercised at the moment or not.

(In the future TLSDESC support may produce similar bundles if it
follows AArch64 implementation. Passes that split bundles would be
just as much of an issue there too).

Cheers.

Tim.




More information about the llvm-commits mailing list