Can we require CFI instructions now?

Currently the only two targets that disable CFI are Hexagon and
powerpc targeting BGP. Is that still correct?

The current integrated assembler is working for PPC, no?

For Hexagon, it has been disabled since the initial checkin. There is
still no support for cfi?

Thanks,
Rafael

From: "Rafael EspĂ­ndola" <rafael.espindola@gmail.com>
To: "LLVM Developers Mailing List" <llvmdev@cs.uiuc.edu>
Cc: tlinth@codeaurora.org, "Hal Finkel" <hfinkel@anl.gov>
Sent: Friday, March 7, 2014 9:27:47 AM
Subject: Can we require CFI instructions now?

Currently the only two targets that disable CFI are Hexagon and
powerpc targeting BGP. Is that still correct?

The current integrated assembler is working for PPC, no?

Yes, and regardless, as far as I'm concerned, the BG/P is now dead.

-Hal

Currently the only two targets that disable CFI are Hexagon and
powerpc targeting BGP. Is that still correct?

The current integrated assembler is working for PPC, no?

Yes, and regardless, as far as I'm concerned, the BG/P is now dead.

Cool. I have changed the BG/P in r203269.

What about hexagon. Do we still need to avoid cfi instructions with it?

Cheers,
Rafael

Hi Rafael,
As far as I know, we've never given CFI any thoughts since the initial implementation of the Hexagon target. I can't really give you an answer right now, but I'll find out and I'll get back to you.

-Krzysztof

What about hexagon. Do we still need to avoid cfi instructions with it?

As far as I know, we've never given CFI any thoughts since the initial implementation of the Hexagon target. I can't really give you an answer right now, but I'll find out and I'll get back to you.

Krzysztof and I discussed this. We're in favor of enabling CFI for Hexagon since it makes sense to require CFI for all targets. We're running some experiments and are discussing this with our assembler team to confirm that we can switch. We'll have the results soonand keep LLVMdev updated.

-Anshu

Awesome!

Thanks,
Rafael

The patch to enable CFI on Hexagon has been posted in llvm-commits.

http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140317/209029.html

-K