[llvm-commits] PATCH: Aligned bundling support in LLVM MC
    Jim Grosbach 
    grosbach at apple.com
       
    Wed Dec 19 16:14:07 PST 2012
    
    
  
Hi Eli,
The code all looks fine to me. Pretty excellent, in fact. I'm OK with this going in. If we change our minds (see below question), we can always back out.
I still have a bit of a funny feeling about doing this sort of thing in the assembler at all, though. It really seems it should be done as part of the target back end using alignment directives in the code stream. That ship may have already sailed, though, if we want to keep compatibility w/ GNU's binutils on these sorts of features. Chris, do you have an opinion on the general direction here?
Thanks,
  Jim
On Dec 14, 2012, at 3:09 PM, Eli Bendersky <eliben at google.com> wrote:
> Hello,
> 
> Following the discussion here:
> http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-December/056754.html
> 
> I'm sending the first set of patches to implement ailgned bundling
> support in LLVM MC. The proposal and implementation are documented
> here:
> 
> https://sites.google.com/a/chromium.org/dev/nativeclient/pnacl/aligned-bundling-support-in-llvm
> 
> There are two patches: one with the implementation and another with the tests.
> 
> Please take a look,
> Eli
> <mc-bundling.impl.1.patch><mc-bundling.tests.1.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
    
    
More information about the llvm-commits
mailing list