[PATCH] Resurrect the assertion removed by r227717
    Jingyue Wu 
    jingyue at google.com
       
    Mon Feb  2 10:12:28 PST 2015
    
    
  
I suspect the issue you saw is a bug in MSVC. There **is** a path from `MDOperand &` to `const MDNode *`. http://llvm.org/docs/doxygen/html/Metadata_8h_source.html#l00547 defines a type caster from `MDOperand &` to `Metadata *`, `Metadata` is a super class of `MDNode`, and finally changing constness is a valid implicit cast.
Anyhow, I noticed `Loop::getLoopID()` returns `MDNode *` instead of `const MDNode *`, and using `MDNode *` seems to work for MSVC. Therefore, I took this approach. Did you confirm it does fix the building error on your end?
http://reviews.llvm.org/D7327
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list